-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merging latest #3
Open
edcdavid
wants to merge
344
commits into
edcdavid:docs
Choose a base branch
from
redhat-best-practices-for-k8s:main
base: docs
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: Jun Chen <[email protected]>
* Get Release version from Git tag * Adding support for building all currently released images (tag based) Co-authored-by: Jun Chen <[email protected]> Co-authored-by: Salaheddine Hamadi <[email protected]>
This will test the partner repo's CRD example in github ci actions.
* Upgrade to Go 1.17.2 * Add in adjustments to workflows from #395 Co-authored-by: Jun Chen <[email protected]>
* Tainted kernel log improvement
* Integrate best practice reference into test catalog * Update * Updated for every test the best practice reference. * Resolved comment of Gonzalo * resolved comments of Jun and Brandon * Resolved comment of Jun * Update resolved comment of Jun * Update resolved comment of Jun Co-authored-by: Shimrit peretz <[email protected]> Co-authored-by: Jun Chen <[email protected]>
…he debug pods managed by a daemon set (#395) * open debug session with all nodes kill pods instead of exit command remove dependency to common from packages fix linter fix mocks and github workflow * address PR comments use daemon set revert changes to github workflow add chroot when required * fix lint * fix lint * fix minikube issue * fix Jun's comments * run go mod tidy * specify mockgen version to fix github action * stop looking for debug pods in minikube * Update go.sum * Revert "specify mockgen version to fix github action" This reverts commit 4744426. * Fix make install-tools * Revert "Revert "specify mockgen version to fix github action"" This reverts commit 3468629. * use new label for debug pods * add node selector to pod debug * add daemonset handler to check the ds availability add common suite for common setup and teardown * update label to match partner repo * update label * add label to node
…0 beta (#397) * Support for more accurate logs in claims and adaptation to Ginkgo V2.0 beta * Update pkg/tnf/handlers/generic/assertion/assertion.go Co-authored-by: Brandon Palm <[email protected]> * Update pkg/tnf/identifier/identifier.go Co-authored-by: Brandon Palm <[email protected]> * PR review * PR review * lint * PR review * PR Review Salah * bumping test-network-function-claim to 1.0.5 * PR review Salah * rest of merge Co-authored-by: Brandon Palm <[email protected]> Co-authored-by: Jun Chen <[email protected]>
The hugepages TC implementation had two main problems: - The /proc/meminfo file shows only the total hugepages for the default size, so in case hugepages count of a different size were configured in the machineconfig, that file won't reflect that count, so it caused the TC to fail. - When MC has hugepages config per numa, this config is set in systemd.units, not in the kernelArguments. With this change, the actual node hugepages values are retrieved by reading these node files with "oc debug": /sys/devices/system/node/node<id>/hugepages/hugepages-<size>-kB/nr_hugepages E.g: /sys/devices/system/node/node0/hugepages/hugepages-2048kB/nr_hugepages In case machineconfig has valid hugepages config in the systemd.units (one unit per numa and size), this info will be compared against node values. Otherwise, kernelArguments will be parsed and compared against the total count of hugepages of that size found in the node. This is because, usually, when no hugepages per numa were explicitly set in the MC, the value is balanced between all the numas in the system. In other words, if system has 2 numas and kernelArguments show 256 hugepages of 2M size, each numa will have 128 hugepages.
* Fix for Ginkgo v2 deprecation warnings and filtering bug * lint * PR Review and bugfix
Fix for duplicate dangling sessions un-expectedly closing
Co-authored-by: Salaheddine Hamadi <[email protected]>
Co-authored-by: Salaheddine Hamadi <[email protected]>
* Fix gitCommit static var * Update run-container.sh Co-authored-by: edcdavid <[email protected]>
* Add SCC capability checks for IPC_LOCK and NET_RAW * fixed make lint * Updated the file privilegedpod.yml * fixed make lint * Resolved comment of Jun Co-authored-by: Shimrit peretz <[email protected]> Co-authored-by: Jun Chen <[email protected]>
Co-authored-by: Jun Chen <[email protected]>
* Expecters Sent/Match debug output only when LOG_LEVEL=trace * Proposal to avoid circular dependencies. Not the most elegant way, though. Co-authored-by: Jun Chen <[email protected]>
* fix helm * fix path
* Avoid panic'ing when helm is not installed. * Fixed unit test. Co-authored-by: Brandon Palm <[email protected]>
* adjusting min coverage to 20% * temp * Removing now extra stateful set step * Revert "temp" This reverts commit e421c3c. Co-authored-by: Brandon Palm <[email protected]>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.44.0 to 1.45.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.44.0...v1.45.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Brandon Palm <[email protected]>
The final timeout for the "oc adm drain" is calculated using this formula: timeout = baseTimeout + (numPodsDeployedOnNode * 1min) where baseTimeout = 5 mins. Finally, the calculated timeout cannot exceed 30mins. Also, I've removed the log.Fatal which resulted in the TNF exiting abruptly when the oc adm drain timed-out, without any possibility to recover, which was leaving the node cordoned. Co-authored-by: Brandon Palm <[email protected]>
* Get all CSV's installPlans As CSV/Operators can be installed through several installPlans, this PR iterates through all of them saving the needed fields. * Use empty string instead of null when indexImage not found. UnitTest has been refactored, but the execCommandOutput function needs an (ugly) small state machine to control the output at each of the calls that are made inside the getCsvInstallPlans functions. * GetCsvInstallPlans refactored. Now it uses smaller functions for each step. Added more UTs.
Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.7.0 to 1.7.1. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](stretchr/testify@v1.7.0...v1.7.1) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…653) * Bug fix: kernel tainted test did not fail on other types of failures * Add counters * Address comments
* Adding data about number of container * Changed function testListenAndDeclared that now run container by container. * Resolved comments of Gonzalo. * Back to the old implementation with little changes * Changed message for the customer Co-authored-by: Shimrit peretz <[email protected]>
Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.18.1 to 1.19.0. - [Release notes](https://github.com/onsi/gomega/releases) - [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md) - [Commits](onsi/gomega@v1.18.1...v1.19.0) --- updated-dependencies: - dependency-name: github.com/onsi/gomega dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Brandon Palm <[email protected]>
Co-authored-by: Salaheddine Hamadi <[email protected]>
+ Some traces fixed in the networking ts.
#666) * Diagnostic mode fix: close debug oc sessions before removing node labels. This commit tries to fix the crash that happens when running the container in diagnostic mode: race condition when the program finishes, as the oc debug session were not closed before the labels are being removed from the nodes. * Revert "Fix timeout (#663)" This reverts commit 8710512. Co-authored-by: Salaheddine Hamadi <[email protected]>
+ Retry interval also increased to 5 secs.
Co-authored-by: Brandon Palm <[email protected]>
Due to the issues on both the original and current implementation, the terminationGracePeriod TC will be removed. A new and improved version of this TC should be done, checking that after the configured (or defaulted) value, the pods were removed without any error code. Co-authored-by: Brandon Palm <[email protected]>
* Solution for issue on Nokia * Fixed the error Co-authored-by: Your Name <[email protected]>
Used in test platform-alteration-sysctl-config
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.45.0 to 1.46.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.45.0...v1.46.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Brandon Palm <[email protected]>
Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.7.1 to 1.7.2. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](stretchr/testify@v1.7.1...v1.7.2) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.46.0 to 1.47.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.46.0...v1.47.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [github.com/hashicorp/go-version](https://github.com/hashicorp/go-version) from 1.4.0 to 1.5.0. - [Release notes](https://github.com/hashicorp/go-version/releases) - [Changelog](https://github.com/hashicorp/go-version/blob/main/CHANGELOG.md) - [Commits](hashicorp/go-version@v1.4.0...v1.5.0) --- updated-dependencies: - dependency-name: github.com/hashicorp/go-version dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.7.2 to 1.7.5. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](stretchr/testify@v1.7.2...v1.7.5) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.